-
-
Notifications
You must be signed in to change notification settings - Fork 20
src/tailscale: remove hair_pinning from ClientSupports model #1265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
It was removed upstream as it's never been used and hasn't been populated in the past year.
|
Warning, @frenck: this is entirely untested. |
|
I also made #1267 to fill in Optional fields with None in case they are missing from the API response. This can prevent future immediate crash of this API if TS removes these from the response. |
|
I've tested this PR and #1267 in real life, they are working. |
|
I also made #1268 to add (nearly) all the missing fields, some of them as optional with default None. |
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions. |
|
not stale |
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions. |
|
not stale |
It was removed upstream as it's never been used and hasn't been populated in the past year.
Updates #1262